Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: set GRUB_CMDLINE_LINUX_DEFAULT #982

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

sticky-note
Copy link
Contributor

@sticky-note sticky-note commented Nov 12, 2024

Summary of Pull Request

I ran into #981 today, it seems to come from GRUB_CMDLINE_LINUX_DEFAULT being set on packer template provisioning via cloud-init installer.
It is set by packer boot_command on kernel cmdline arguments and tell cloud-init to use a specific DataSource (nocloud-net or nocloud) and the installer persists these arguments on grub configuration in /etc/default/grub.
It it the reason why cloud-init does not read VMware DataSource.

I ended up with this Pull Request to restablish an empty GRUB_CMDLINE_LINUX_DEFAULT as late_command

It works on my side

Tested on Ubuntu 22.04 LTS

Hope it'll help someone.

Type of Pull Request

  • This is a bugfix. type/bug
  • This is an enhancement or feature. type/feature or type/enhancement
  • This is a documentation update. type/docs
  • This is a refactoring update. type/refactor
  • This is a chore. type/chore
  • This is something else.
    Please describe:

Related to Existing Issues

Closes #981

Test and Documentation Coverage

  • Tests have been completed.
  • Documentation has been added or updated.

Breaking Changes?

  • Yes, there are breaking changes.
  • No, there are no breaking changes.

@tenthirtyam
Copy link
Collaborator

@sticky-note - please amend your commit with the required sign-off. Once updated, we'll be able to review and proceed.

Thanks!

Ryan Johnson
Distinguished Engineer, VMware by Broadcom

@tenthirtyam tenthirtyam modified the milestones: Backlog, v0.22.0 Nov 21, 2024
@tenthirtyam
Copy link
Collaborator

@sticky-note - if you are uncomfortable adding the sign-off, please let me know and I'll add the fix seperately.

@tenthirtyam tenthirtyam changed the title fix(ubuntu): restablish empty default grub cmdline after cloud-init install fix: set GRUB_CMDLINE_LINUX_DEFAULT Dec 2, 2024
Reestablish an empty `GRUB_CMDLINE_LINUX_DEFAULT` as `late_command`.

Signed-off-by: sticky-note <[email protected]>
Copy link
Collaborator

@tenthirtyam tenthirtyam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tenthirtyam tenthirtyam merged commit 4fefa05 into vmware-samples:develop Dec 2, 2024
1 check passed
tenthirtyam pushed a commit that referenced this pull request Dec 2, 2024
Reestablish an empty `GRUB_CMDLINE_LINUX_DEFAULT` as `late_command`.

Signed-off-by: sticky-note <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to assign static IP to Ubuntu 22.04 image result with cloud-init
2 participants